Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support RegularExpression for path matching #10717

Merged
merged 2 commits into from
May 23, 2024

Conversation

dmavrommatis
Copy link
Contributor

What does this PR do?

Pull request introduced support for path matching of type RegularExpression from Gateway API.

Motivation

Moving from traefik/IngressRoute to gateway/HttpRoute is not possible since we are using a lot of PathRegexp which is not yet supported.

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

@kevinpollet
Copy link
Member

Hello @dmavrommatis and thanks for your contribution,

As this is an enhancement, please could you rebase this pull request on master?

@dmavrommatis
Copy link
Contributor Author

Hello @dmavrommatis and thanks for your contribution,

As this is an enhancement, please could you rebase this pull request on master?

Changed the target branch to master and rebased. Previously I chose v3.0 branch because of the PR guide advising to use the v3.0 branch for enchancements.

Enhancements:
- for Traefik v2: we only accept bug fixes
- for Traefik v3: use branch v3.0

@ldez ldez added this to the next milestone May 14, 2024
@rtribotte rtribotte self-assigned this May 16, 2024
@kevinpollet kevinpollet self-assigned this May 16, 2024
@kevinpollet kevinpollet force-pushed the add-httproute-regex-pathmatch branch from 8245755 to 749930d Compare May 23, 2024 15:03
Copy link
Member

@kevinpollet kevinpollet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 👍

Copy link
Member

@rtribotte rtribotte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Contributor

@lbenguigui lbenguigui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@traefiker traefiker merged commit 6e61fe0 into traefik:master May 23, 2024
23 checks passed
@dmavrommatis dmavrommatis deleted the add-httproute-regex-pathmatch branch May 24, 2024 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants